-
-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[web-app] Move generic cookie steps to a common parent class #5604
[web-app] Move generic cookie steps to a common parent class #5604
Conversation
Qodana for JVM2 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
@@ -66,7 +66,7 @@ | |||
|
|||
<bean class="org.vividus.ui.web.playwright.screenshot.PlaywrightScreenshotTaker" /> | |||
|
|||
<bean id="cookieSteps" class="org.vividus.ui.web.playwright.steps.CookieSteps" /> | |||
<bean id="cookieSteps" class="org.vividus.ui.web.playwright.steps.PlaywrightCookieSteps" /> |
Check failure
Code scanning / QDJVM
Incorrect constructor injection in XML Spring bean Error
| PlaywrightCookieSteps(...): | | Bean: |
| UiContext uiContext | | Autowired: null(UiContext) |
| CookieManager<Cookie> cookieManager | | Autowired: null(PlaywrightCookieManager) |
| VariableContext variableContext | | ??? |
| JsonUtils jsonUtils | | ??? |
| ISoftAssert softAssert | | ??? |
@@ -179,7 +179,7 @@ | |||
<bean id="alertSteps" class="org.vividus.steps.ui.web.AlertSteps" /> | |||
<bean id="buttonSteps" class="org.vividus.steps.ui.web.RadioButtonSteps" /> | |||
<bean id="codeSteps" class="org.vividus.steps.ui.web.CodeSteps" /> | |||
<bean id="cookieSteps" class="org.vividus.steps.ui.web.CookieSteps" /> | |||
<bean id="cookieSteps" class="org.vividus.steps.ui.web.SeleniumCookieSteps" /> |
Check failure
Code scanning / QDJVM
Incorrect constructor injection in XML Spring bean Error
| SeleniumCookieSteps(...): | | Bean: |
| IWebDriverProvider webDriverProvider | | ??? |
| CookieManager<Cookie> cookieManager | | Autowired: cookieManager(WebDriverCookieManager) |
| INavigateActions navigateActions | | Autowired: navigateActions(NavigateActions) |
| VariableContext variableContext | | ??? |
| JsonUtils jsonUtils | | ??? |
| ISoftAssert softAssert | | ??? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5604 +/- ##
============================================
- Coverage 97.69% 97.69% -0.01%
+ Complexity 7372 7366 -6
============================================
Files 987 988 +1
Lines 20675 20664 -11
Branches 1353 1351 -2
============================================
- Hits 20199 20188 -11
Misses 364 364
Partials 112 112 ☔ View full report in Codecov by Sentry. |
6fe2505
to
86bada8
Compare
86bada8
to
d4f69bf
Compare
Quality Gate passedIssues Measures |
No description provided.